Skip to content

Comments

[8.19] Change GeoIpCache and EnrichCache to LongAdder (#132922)#133249

Merged
elasticsearchmachine merged 1 commit intoelastic:8.19from
szybia:backport/8.19/pr-132922
Aug 20, 2025
Merged

[8.19] Change GeoIpCache and EnrichCache to LongAdder (#132922)#133249
elasticsearchmachine merged 1 commit intoelastic:8.19from
szybia:backport/8.19/pr-132922

Conversation

@szybia
Copy link
Contributor

@szybia szybia commented Aug 20, 2025

Backports the following commits to 8.19:

From the javadocs:

> [`LongAdder`] is usually preferable to `AtomicLong` when multiple threads update a common sum that is used for purposes such as collecting statistics, not for fine-grained synchronization control.

Since we largely write stats and read very infrequently, let's avoid any lock-contention possibilities by switching to the more appropriate data type.
@szybia szybia added :Distributed/Ingest Node Execution or management of Ingest Pipelines >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Data Management (obsolete) DO NOT USE. This team no longer exists. labels Aug 20, 2025
@elasticsearchmachine elasticsearchmachine merged commit 54af22b into elastic:8.19 Aug 20, 2025
22 checks passed
@szybia szybia deleted the backport/8.19/pr-132922 branch August 20, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Distributed/Ingest Node Execution or management of Ingest Pipelines Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v8.19.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants